-
Notifications
You must be signed in to change notification settings - Fork 21.6k
ethclient: add SubscribeTransactionReceipts #32869
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| // TransactionReceiptsQuery defines criteria for transaction receipts subscription. | ||
| // If TransactionHashes is empty, receipts for all transactions included in new blocks will be delivered. | ||
| // Otherwise, only receipts for the specified transactions will be delivered. | ||
| type TransactionReceiptsQuery struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can duplicate this type in ethclient. Feels weird to have a struct in the interfaces file!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see there are other structs in this file, for example, FilterQuery/CallMsg. How should I make the change more appropriately?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough, forgot we have the query types also here.
s1na
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Add `SubscribeTransactionReceipts` for ethclient. This is a complement to ethereum#32697.
Add `SubscribeTransactionReceipts` for ethclient. This is a complement to ethereum#32697.
Add `SubscribeTransactionReceipts` for ethclient. This is a complement to ethereum#32697.
Add
SubscribeTransactionReceiptsfor ethclient. This is a complement to #32697.